Skip to content

UofT-DSI | Python - Assignment 1#1

Open
r0bay wants to merge 1 commit into
mainfrom
assignment-1
Open

UofT-DSI | Python - Assignment 1#1
r0bay wants to merge 1 commit into
mainfrom
assignment-1

Conversation

@r0bay
Copy link
Copy Markdown
Owner

@r0bay r0bay commented Apr 20, 2026

1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

I built an anagram checker that takes two words and tells you if they’re anagrams or not. Then I updated it so you can choose whether it should care about uppercase and lowercase or ignore it

What did you learn from the changes you have made?

I learned that instead of checking every letter one by one, you can just sort both words and compare them

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

I also thought about counting how many times each letter shows up in both words and comparing that, but sorting felt simpler so I went with that

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

Sometimes I forgot to put colons at the end of the functions and got errors when running the code. I fixed that by checking the error messages and going back to my notes

How were these changes tested?

I tested it by running the cells in the notebook and using the play button to make sure there were no errors. I also checked the outputs using the example words to confirm it was returning the correct true or false results

A reference to a related issue in your repository (if applicable)

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@KasraVakiloroayaei KasraVakiloroayaei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A1: Complete
Good work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants